Drop use of gail_return_if_fail
authorMatthias Clasen <mclasen@redhat.com>
Wed, 29 Jun 2011 02:48:37 +0000 (22:48 -0400)
committerMatthias Clasen <mclasen@redhat.com>
Tue, 5 Jul 2011 20:08:54 +0000 (16:08 -0400)
gtk/a11y/gailmenuitem.c
gtk/a11y/gailwindow.c

index 73889c7cecc9c9a48a7ae0d82e646afdef90b7c8..04e1c64e37b474a00156116981cc86007229eec6 100644 (file)
@@ -22,7 +22,6 @@
 #include <gdk/gdkkeysyms.h>
 #include "gailmenuitem.h"
 #include "gailsubmenuitem.h"
-#include <libgail-util/gailmisc.h>
 
 #define KEYBINDING_SEPARATOR ";"
 
index 6cf10665157e7502eeef7c63cf858f8657e86c31..80d2732e32432e8fe135c33a1a4c8083a2ed69c9 100644 (file)
@@ -25,7 +25,6 @@
 
 #include "gailwindow.h"
 #include "gailtoplevel.h"
-#include "gail-private-macros.h"
 
 enum {
   ACTIVATE,
@@ -191,9 +190,8 @@ gail_window_real_initialize (AtkObject *obj,
   /*
    * A GailWindow can be created for a GtkHandleBox or a GtkWindow
    */
-  if (!GTK_IS_WINDOW (widget) &&
-      !GTK_IS_HANDLE_BOX (widget))
-    gail_return_if_fail (FALSE);
+  if (!GTK_IS_WINDOW (widget) && !GTK_IS_HANDLE_BOX (widget))
+    return;
 
   ATK_OBJECT_CLASS (gail_window_parent_class)->initialize (obj, data);
 
@@ -277,13 +275,8 @@ gail_window_get_name (AtkObject *accessible)
       GtkWidget* widget = gtk_accessible_get_widget (GTK_ACCESSIBLE (accessible));
 
       if (widget == NULL)
-        /*
-         * State is defunct
-         */
         return NULL;
 
-      gail_return_val_if_fail (GTK_IS_WIDGET (widget), NULL);
-
       if (GTK_IS_WINDOW (widget))
         {
           GtkWindow *window = GTK_WINDOW (widget);
@@ -345,13 +338,8 @@ gail_window_get_index_in_parent (AtkObject *accessible)
   gint index = -1;
 
   if (widget == NULL)
-    /*
-     * State is defunct
-     */
     return -1;
 
-  gail_return_val_if_fail (GTK_IS_WIDGET (widget), -1);
-
   index = ATK_OBJECT_CLASS (gail_window_parent_class)->get_index_in_parent (accessible);
   if (index != -1)
     return index;
@@ -399,13 +387,8 @@ gail_window_ref_relation_set (AtkObject *obj)
   AtkRelation* relation;
   GtkWidget *current_widget;
 
-  gail_return_val_if_fail (GAIL_IS_WIDGET (obj), NULL);
-
   widget = gtk_accessible_get_widget (GTK_ACCESSIBLE (obj));
   if (widget == NULL)
-    /*
-     * State is defunct
-     */
     return NULL;
 
   relation_set = ATK_OBJECT_CLASS (gail_window_parent_class)->ref_relation_set (obj);
@@ -558,13 +541,8 @@ gail_window_get_extents (AtkComponent  *component,
   gint x_toplevel, y_toplevel;
 
   if (widget == NULL)
-    /*
-     * State is defunct
-     */
     return;
 
-  gail_return_if_fail (GTK_IS_WINDOW (widget));
-
   if (!gtk_widget_is_toplevel (widget))
     {
       AtkComponentIface *parent_iface;
@@ -605,13 +583,8 @@ gail_window_get_size (AtkComponent *component,
   GdkRectangle rect;
 
   if (widget == NULL)
-    /*
-     * State is defunct
-     */
     return;
 
-  gail_return_if_fail (GTK_IS_WINDOW (widget));
-
   if (!gtk_widget_is_toplevel (widget))
     {
       AtkComponentIface *parent_iface;
@@ -934,9 +907,7 @@ init_gail_screen (GdkScreen *screen,
 static GailScreenInfo *
 get_screen_info (GdkScreen *screen)
 {
-  int screen_n;
-
-  gail_return_val_if_fail (GDK_IS_SCREEN (screen), NULL);
+  gint screen_n;
 
   screen_n = gdk_screen_get_number (screen);
 
@@ -964,11 +935,10 @@ get_window_zorder (GdkWindow *window)
   int             zorder;
   int             w_desktop;
 
-  gail_return_val_if_fail (GDK_IS_WINDOW (window), -1);
-
   info = get_screen_info (gdk_window_get_screen (window));
 
-  gail_return_val_if_fail (info->stacked_windows != NULL, -1);
+  if (info->stacked_windows == NULL)
+    return -1;
 
   xid = GDK_WINDOW_XID (window);
 
@@ -1007,13 +977,8 @@ gail_window_get_mdi_zorder (AtkComponent *component)
   GtkWidget *widget = gtk_accessible_get_widget (GTK_ACCESSIBLE (component));
 
   if (widget == NULL)
-    /*
-     * State is defunct
-     */
     return -1;
 
-  gail_return_val_if_fail (GTK_IS_WINDOW (widget), -1);
-
   return get_window_zorder (gtk_widget_get_window (widget));
 }
 
@@ -1025,13 +990,8 @@ gail_window_get_mdi_zorder (AtkComponent *component)
   GtkWidget *widget = gtk_accessible_get_widget (GTK_ACCESSIBLE (component));
 
   if (widget == NULL)
-    /*
-     * State is defunct
-     */
     return -1;
 
-  gail_return_val_if_fail (GTK_IS_WINDOW (widget), -1);
-
   return 0;                    /* Punt, FIXME */
 }
 
@@ -1043,13 +1003,8 @@ gail_window_get_mdi_zorder (AtkComponent *component)
   GtkWidget *widget = gtk_accessible_get_widget (GTK_ACCESSIBLE (component));
 
   if (widget == NULL)
-    /*
-     * State is defunct
-     */
     return -1;
 
-  gail_return_val_if_fail (GTK_IS_WINDOW (widget), -1);
-
   return 0;                    /* Punt, FIXME */
 }